(AB-506233) Clarify note in Remove-Item#12707
Conversation
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
There was a problem hiding this comment.
Pull request overview
This PR clarifies documentation for the Remove-Item cmdlet by expanding a brief note about a behavior change in Windows version 1909. The original note simply stated "This behavior was fixed in Windows versions 1909 and up" without explaining what the fix enables or how users can benefit from it.
Changes:
- Expands the note to explain that starting in Windows 1909,
Get-ChildItemcan use-Path *.csv -Recursedirectly instead of requiring the workaround-Path * -Include *.csv -Recurse - Adds side-by-side code examples showing both approaches with explanatory comments
- Updates the
ms.datefield to reflect the documentation change date
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| reference/7.6/Microsoft.PowerShell.Management/Remove-Item.md | Updates documentation note and date for PowerShell 7.6 reference |
| reference/7.5/Microsoft.PowerShell.Management/Remove-Item.md | Updates documentation note and date for PowerShell 7.5 reference |
| reference/7.4/Microsoft.PowerShell.Management/Remove-Item.md | Updates documentation note and date for PowerShell 7.4 reference |
| reference/5.1/Microsoft.PowerShell.Management/Remove-Item.md | Updates documentation note and date for PowerShell 5.1 reference |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Learn Build status updates of commit 3ca95fd: ✅ Validation status: passed
For more details, please refer to the build report. |
Prior to this change, the `Remove-Item` documentation included a brief note for example 4 meant to indicate that the unusual construction of the command in the example: ```powershell Get-ChildItem * -Include *.csv -Recurse | Remove-Item ``` Is no longer required in Windows version 1909 and later. This change: - Expands on the note to clarify the change in behavior and show how a user can write a more idiomatic statement in later versions of Windows to fulfill the same purpose. - Fixes AB#506233
sdwheeler
force-pushed
the
ab-506233/main/clarify-note-remove-item
branch
from
3ca95fd to
2260454
Compare
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 2260454: ✅ Validation status: passed
For more details, please refer to the build report. |
PR Summary
Prior to this change, the
Remove-Itemdocumentation included a brief note for example 4 meant to indicate that the unusual construction of the command in the example:Is no longer required in Windows version 1909 and later.
This change:
PR Checklist